-
Notifications
You must be signed in to change notification settings - Fork 3.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(docker): separation from autoware-universe
stage to autoware-universe-sensing-perception
stage
#5164
Conversation
Signed-off-by: Yutaka Kondo <yutaka.kondo@youtalk.jp>
Signed-off-by: Yutaka Kondo <yutaka.kondo@youtalk.jp>
Signed-off-by: Yutaka Kondo <yutaka.kondo@youtalk.jp>
This reverts commit 789ee21.
Signed-off-by: Yutaka Kondo <yutaka.kondo@youtalk.jp>
autoware-universe
stage to autoware-universe-sensing-perception
stageautoware-universe
stage to autoware-universe-sensing-perception
stage
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, but we should find a way to simplify and clarify the dockerfile after completing all the intermediary stages. Since it is getting harder to read on each iteration.
I know. That is why I proposed the base image to split the Dockerfiles. However you said this is also complex... https://github.com/orgs/autowarefoundation/discussions/5110#discussioncomment-10468337
|
…universe-sensing-perception` stage (autowarefoundation#5164) * split depend stage Signed-off-by: Yutaka Kondo <yutaka.kondo@youtalk.jp> * add s/p stage Signed-off-by: Yutaka Kondo <yutaka.kondo@youtalk.jp> * fix Signed-off-by: Yutaka Kondo <yutaka.kondo@youtalk.jp> * rename Signed-off-by: Yutaka Kondo <yutaka.kondo@youtalk.jp> * fix Signed-off-by: Yutaka Kondo <yutaka.kondo@youtalk.jp> * revert rename Signed-off-by: Yutaka Kondo <yutaka.kondo@youtalk.jp> * Revert "revert rename" This reverts commit 789ee21. * simplify rosdep-depend Signed-off-by: Yutaka Kondo <yutaka.kondo@youtalk.jp> --------- Signed-off-by: Yutaka Kondo <yutaka.kondo@youtalk.jp>
…universe-sensing-perception` stage (#5164) * split depend stage Signed-off-by: Yutaka Kondo <yutaka.kondo@youtalk.jp> * add s/p stage Signed-off-by: Yutaka Kondo <yutaka.kondo@youtalk.jp> * fix Signed-off-by: Yutaka Kondo <yutaka.kondo@youtalk.jp> * rename Signed-off-by: Yutaka Kondo <yutaka.kondo@youtalk.jp> * fix Signed-off-by: Yutaka Kondo <yutaka.kondo@youtalk.jp> * revert rename Signed-off-by: Yutaka Kondo <yutaka.kondo@youtalk.jp> * Revert "revert rename" This reverts commit 789ee21. * simplify rosdep-depend Signed-off-by: Yutaka Kondo <yutaka.kondo@youtalk.jp> --------- Signed-off-by: Yutaka Kondo <yutaka.kondo@youtalk.jp>
* rename stages Signed-off-by: Yutaka Kondo <yutaka.kondo@youtalk.jp> * update docker-bake Signed-off-by: Yutaka Kondo <yutaka.kondo@youtalk.jp> * update actions Signed-off-by: Yutaka Kondo <yutaka.kondo@youtalk.jp> * fix target Signed-off-by: Yutaka Kondo <yutaka.kondo@youtalk.jp> * fix inherits Signed-off-by: Yutaka Kondo <yutaka.kondo@youtalk.jp> * feat(docker): separation from `autoware-universe` stage to `autoware-universe-sensing-perception` stage (#5164) * split depend stage Signed-off-by: Yutaka Kondo <yutaka.kondo@youtalk.jp> * add s/p stage Signed-off-by: Yutaka Kondo <yutaka.kondo@youtalk.jp> * fix Signed-off-by: Yutaka Kondo <yutaka.kondo@youtalk.jp> * rename Signed-off-by: Yutaka Kondo <yutaka.kondo@youtalk.jp> * fix Signed-off-by: Yutaka Kondo <yutaka.kondo@youtalk.jp> * revert rename Signed-off-by: Yutaka Kondo <yutaka.kondo@youtalk.jp> * Revert "revert rename" This reverts commit 789ee21. * simplify rosdep-depend Signed-off-by: Yutaka Kondo <yutaka.kondo@youtalk.jp> --------- Signed-off-by: Yutaka Kondo <yutaka.kondo@youtalk.jp> * fix rebase Signed-off-by: Yutaka Kondo <yutaka.kondo@youtalk.jp> * rename stages Signed-off-by: Yutaka Kondo <yutaka.kondo@youtalk.jp> * fix bake Signed-off-by: Yutaka Kondo <yutaka.kondo@youtalk.jp> * fix dockerfile Signed-off-by: Yutaka Kondo <yutaka.kondo@youtalk.jp> * fix merge Signed-off-by: Yutaka Kondo <yutaka.kondo@youtalk.jp> * add universe-sensing-perception stage Signed-off-by: Yutaka Kondo <yutaka.kondo@youtalk.jp> * update config Signed-off-by: Yutaka Kondo <yutaka.kondo@youtalk.jp> --------- Signed-off-by: Yutaka Kondo <yutaka.kondo@youtalk.jp>
Description
Resolved #5079
This PR separates from the
autoware-universe
stage to theautoware-universe-sensing-perception
stage, which builds only thesensing
andperception
components.Subsequent PRs will similarly add stages divided by components.
Tests performed
Not applicable.
Effects on system behavior
Not applicable.
Interface changes
Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.
In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.
Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.
After all checkboxes are checked, anyone who has write access can merge the PR.